Skip to content

RULEAPI-770 Fix bug due to multiple sonarpedia.json files #3983

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 11, 2024

Conversation

johann-beleites-sonarsource
Copy link
Contributor

When there are multiple sonarpedia.json files for the same language, the rules picked up for the next sonarpedia.json file will override the ones picked up previously. With this fix, the rules are not overwritten but all are collected, so one repository can have multiple sonarpedia.json files for the same language.

@johann-beleites-sonarsource johann-beleites-sonarsource added the rspec system Related to handling of rspecs (scripts, doc...) label Jun 6, 2024
@johann-beleites-sonarsource johann-beleites-sonarsource force-pushed the jb/RULEAPI-770 branch 2 times, most recently from 9306f34 to 19a94bb Compare June 7, 2024 10:09
When there are multiple sonarpedia.json files for the same language, the rules
picked up for the next sonarpedia.json file will override the ones picked up
previously. With this fix, the rules are not overwritten but all are collected,
so one repository can have multiple sonarpedia.json files for the same language.
@johann-beleites-sonarsource johann-beleites-sonarsource force-pushed the jb/RULEAPI-770 branch 2 times, most recently from 773ffd4 to 87eaf86 Compare June 10, 2024 15:17
@johann-beleites-sonarsource johann-beleites-sonarsource marked this pull request as ready for review June 10, 2024 16:01
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change is functionally good for me.
I'm no expert in Python, but I think we can make it simpler by using some nice Dictionary features.

Copy link

Quality Gate passed Quality Gate passed for 'rspec-frontend'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link

@johann-beleites-sonarsource johann-beleites-sonarsource merged commit e2c4c4b into master Jun 11, 2024
10 of 11 checks passed
@johann-beleites-sonarsource johann-beleites-sonarsource deleted the jb/RULEAPI-770 branch June 11, 2024 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rspec system Related to handling of rspecs (scripts, doc...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants